Skip to content

🤖 feat: add Argo CD single-apply ApplicationSet example#75

Merged
ThomasK33 merged 2 commits into
mainfrom
argocd-deploy-ab27
Feb 13, 2026
Merged

🤖 feat: add Argo CD single-apply ApplicationSet example#75
ThomasK33 merged 2 commits into
mainfrom
argocd-deploy-ab27

Conversation

@ThomasK33
Copy link
Copy Markdown
Member

Summary

Add a new Argo CD example that lets users bootstrap CloudNativePG + coder-k8s + PostgreSQL + CoderControlPlane from a single ApplicationSet manifest.

Background

The repository already had a manual CloudNativePG example, but GitOps users had to perform several separate install steps. This change adds a one-apply Argo CD flow aligned with that example so evaluator clusters can stand up quickly.

Implementation

  • Added examples/argocd/ with:
    • applicationset.yaml (single resource to apply)
    • child Applications for:
      • CloudNativePG operator (Helm source)
      • coder-k8s operator stack (config/crd/bases, config/rbac, deploy)
      • CloudNativePG-backed Coder resources (examples/cloudnativepg)
  • Added sync-wave ordering between child Applications (wave 0 operators, wave 1 app resources).
  • Added resource-level Argo sync-wave annotations in examples/cloudnativepg/:
    • namespace wave 0
    • CNPG Cluster wave 1
    • CoderControlPlane wave 2
  • Updated README docs:
    • top-level README.md examples list now includes examples/argocd/
    • examples/cloudnativepg/README.md now documents shared Argo sync-wave behavior

Validation

  • make verify-vendor
  • make test
  • make build
  • make lint
  • make docs-check
  • Parsed changed YAML files with yaml.safe_load_all to verify syntax.

Risks

  • The example references targetRevision: main for repository-backed Applications, which is intentionally convenient for demos but not pinned for reproducible production installs.
  • The CloudNativePG chart target revision remains flexible by default; users should pin versions for stricter reproducibility.

Generated with mux • Model: openai:gpt-5.3-codex • Thinking: xhigh • Cost: $0.56

Add an Argo CD ApplicationSet example that bootstraps CloudNativePG, coder-k8s, a PostgreSQL cluster, and a CoderControlPlane from one manifest.

Also add Argo sync-wave annotations to the existing CloudNativePG example manifests so they apply in dependency order, and document both examples in README files.

---

_Generated with [`mux`](https://github.com/coder/mux) • Model: `openai:gpt-5.3-codex` • Thinking: `xhigh` • Cost: `$0.56`_

<!-- mux-attribution: model=openai:gpt-5.3-codex thinking=xhigh costs=0.56 -->
@ThomasK33
Copy link
Copy Markdown
Member Author

@codex review

Please review this Argo CD example addition for correctness and deployment ordering.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f35546fcf8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread examples/argocd/apps/02-coder-cloudnativepg.yaml Outdated
Comment thread examples/argocd/apps/00-cnpg-operator.yaml Outdated
Refactor the Argo CD example so the ApplicationSet generates one multi-source Application instead of a parent app that applies child Application resources.

This avoids cross-Application wave ordering ambiguity, adds a cascading resources finalizer for cleanup, and keeps dependency ordering at the resource level via sync waves.

---

_Generated with [`mux`](https://github.com/coder/mux) • Model: `openai:gpt-5.3-codex` • Thinking: `xhigh` • Cost: `$0.56`_

<!-- mux-attribution: model=openai:gpt-5.3-codex thinking=xhigh costs=0.56 -->
@ThomasK33
Copy link
Copy Markdown
Member Author

@codex review

Addressed feedback by refactoring the Argo example to a single multi-source generated Application (no app-of-apps wave dependency), and added a resources finalizer for cascading cleanup.

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ThomasK33 ThomasK33 added this pull request to the merge queue Feb 13, 2026
@ThomasK33
Copy link
Copy Markdown
Member Author

Merged via the queue into main with commit 687e6bd Feb 13, 2026
8 checks passed
@ThomasK33 ThomasK33 deleted the argocd-deploy-ab27 branch February 13, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant